Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[kbn/es-archiver] move to a package #72318

Merged
merged 8 commits into from
Jul 22, 2020

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jul 17, 2020

As a part of moving the es-archiver to a more modern place I'd like to move the es_archiver logic to a package that is isolated from the rest of Kibana.

This PR uses the new RunWithCommands helper from @kbn/dev-utils added in #72311

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.9.0 v7.8.1 labels Jul 17, 2020
@spalger spalger force-pushed the implement/es-archiver-package branch from 42422fe to 6826eec Compare July 17, 2020 22:04
* under the License.
*/

export * from '../../../../src/legacy/utils/streams';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is temporary until I can move esArchiver off of node streams.

@spalger spalger marked this pull request as ready for review July 17, 2020 22:06
@spalger spalger requested review from a team as code owners July 17, 2020 22:06
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@dmlemeshko
Copy link
Member

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM. I successfully verified save, load and --help flag.

Copy link
Member

@mistic mistic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@spalger spalger merged commit 9655c87 into elastic:master Jul 22, 2020
@spalger spalger deleted the implement/es-archiver-package branch July 22, 2020 15:02
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 22, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	.github/CODEOWNERS
#	src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 22, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	.github/CODEOWNERS
#	src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
@spalger spalger removed the v7.8.1 label Jul 22, 2020
spalger pushed a commit that referenced this pull request Jul 22, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	.github/CODEOWNERS
#	src/dev/code_coverage/ingest_coverage/team_assignment/ingestion_pipeline_painless.json
spalger added a commit that referenced this pull request Jul 22, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.9.0 v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants